Skip to content

Conversation

@ammar-elsabe
Copy link

Description

Closes: #25420

  • Changed the keeper type in x/epochs ModuleOutputs to be a pointer to the keeper stored in the AppModule
    • EpochsKeeper's type in simapp/app_di.go was updated to match
  • Added a test to verify depinject finds and invokes setting hooks
  • Documented how to wire x/epochs in the README

@ammar-elsabe ammar-elsabe force-pushed the main branch 6 times, most recently from 735c574 to f32e9cb Compare October 11, 2025 09:02
@codecov
Copy link

codecov bot commented Oct 20, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.28%. Comparing base (9578e44) to head (0f1cb89).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main   #25425      +/-   ##
==========================================
+ Coverage   70.00%   70.28%   +0.28%     
==========================================
  Files         827      822       -5     
  Lines       54070    53627     -443     
==========================================
- Hits        37851    37693     -158     
+ Misses      16219    15934     -285     
Files with missing lines Coverage Δ
simapp/app_di.go 78.89% <ø> (ø)
x/epochs/depinject.go 86.36% <100.00%> (-4.55%) ⬇️
x/epochs/module.go 63.26% <100.00%> (ø)

... and 9 files with indirect coverage changes

Impacted file tree graph

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ammar-elsabe ammar-elsabe force-pushed the main branch 2 times, most recently from 8dc36f3 to ecfea47 Compare October 21, 2025 06:13
@ammar-elsabe
Copy link
Author

@aljo242 fixed the build

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

InvokeSetHooks in x/epochs is called with a nil keeper

2 participants